Dashboard search, display most recent dashboards first, display who created dashboard#259
Conversation
…reated dashboard Fix #253
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5acddb8729
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR enhances the Dashboards UX and metadata by adding client-side search, ordering dashboards with the most recently created first, and persisting/displaying additional dashboard metadata (creator info and last evaluation time). It also adds/updates tests to validate the new sorting and persistence behavior.
Changes:
- Add dashboard list search UI with match highlighting and empty-state messaging.
- Sort dashboards newest-first and persist
lastEvaluatedAton evaluation completion. - Store creator metadata (
createdById+createdBy) when dashboards are created, with new test coverage.
Reviewed changes
Copilot reviewed 11 out of 11 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| test/Dashboard.getDashboards.test.js | Adds tests for newest-first ordering, legacy createdAt derivation, and creator persistence. |
| test/Dashboard.getDashboard.results.test.js | Verifies lastEvaluatedAt is persisted to the dashboard when evaluations complete. |
| frontend/src/dashboards/dashboards.js | Adds search state, filtering, highlighting helpers, date/user formatting, and inserts new dashboards at the top. |
| frontend/src/dashboards/dashboards.html | Adds the search input, “no matches” state, switches list rendering to filtered dashboards, and displays metadata fields. |
| express.js | Propagates initiating user info into request internals for downstream actions. |
| backend/next.js | Propagates initiating user info into action params for Next.js deployments. |
| backend/netlify.js | Propagates initiating user info into action params for Netlify deployments. |
| backend/db/dashboardSchema.js | Adds creator + evaluation fields and attempts legacy createdAt backfill via a query post-hook. |
| backend/actions/Dashboard/getDashboards.js | Sorts dashboards by newest first and returns lean objects. |
| backend/actions/Dashboard/getDashboard.js | Updates dashboard lastEvaluatedAt when an evaluation finishes. |
| backend/actions/Dashboard/createDashboard.js | Persists creator fields when creating a new dashboard. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fix #253